Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Fix] manifest 및 og tag link 수정 #419

Merged
merged 2 commits into from
Aug 17, 2024
Merged

[Fix] manifest 및 og tag link 수정 #419

merged 2 commits into from
Aug 17, 2024

Conversation

lydiacho
Copy link
Member

Related Issue : Closes #418


🧑‍🎤 Summary

  • index.html 하드코딩 수정 과정에서 manifest 파일명을 잘못써서 수정했습니다
  • og tag Link를 full url로 수정해주었습니다.

🧑‍🎤 Screenshot & Comment

image

현재 위와 같이 다른 Og는 잘 뜨는데, 이미지만 안뜨고 있는걸 확인할 수 있어요.
og의 경우 public 폴더 하의 에셋들을 사용할 때, 경로를 작성하면 안되고 도메인까지 붙인 full url을 넣어줘야 정상작동합니다.
따라서 url로 고쳐주었어요!
배포해서 확인해보고 잘 되면 SOPT 레포에도 반영할 예정입니다

Copy link

height bot commented Aug 17, 2024

Link Height tasks by mentioning a task ID in the pull request title or commit messages, or description and comments with the keyword link (e.g. "Link T-123").

💡Tip: You can also use "Close T-X" to automatically close a task when the pull request is merged.

@lydiacho lydiacho merged commit 506a3e8 into develop Aug 17, 2024
1 check passed
@lydiacho lydiacho deleted the fix/#418_manifest branch August 17, 2024 15:24

<meta property="og:title" content="SOPT makers 모집 지원하기" />
<meta property="og:description" content="SOPT makers의 신입 기수 모집페이지입니다." />
<meta property="og:site_name" content="SOPT makers 리크루팅" />
<meta property="og:url" content="https://recruiting.sopt.org" />

<meta property="og:image" content="/makersOg.png" />
<meta property="og:image" content="https://recruiting.sopt.org/makersOg.png" />
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

헉스 아니에요!

public에 있는건 상대경로로 설정해도 정상작동합니다!!

현재 ogImage가 보여지고 있지 않은 이유는 makersOg.png 파일 이름이 maekrsOg로 오타가 나서 그래요ㅜㅜㅜ
다른 부분도 다 full url 말고 / 이후 부분만 입력해도 다 정상 작동합니다 :)

근데 사실 위와 같이 표현해도 문제될 건 없으니 이대로 가도 좋습니다~~

고생하셨습니다ㅜ!!

lydiacho added a commit to sopt-makers/sopt-recruit-frontend that referenced this pull request Aug 17, 2024
* [Fix] manifest 및 og tag link 수정 (sopt-makers#419)

* fix: manifest 파일명

* fix: og tag link full url로 수정

* [Fix] Og 이미지 파일명 오탈자 수정 (sopt-makers#421)

* fix: full url 제거 확인

* fix: ogimage 파일명 수정
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Fix] mainfest 파일명 수정
2 participants